Skip to content

ProjectModule3SpaceQuest#1

Open
Roman-M-git wants to merge 1 commit into
masterfrom
reviewModule3
Open

ProjectModule3SpaceQuest#1
Roman-M-git wants to merge 1 commit into
masterfrom
reviewModule3

Conversation

@Roman-M-git

Copy link
Copy Markdown
Owner

Здравствуйте!

Прошу проверить мою работу за 3-й модуль.

Роман

@sproshchaev sproshchaev left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Принято. Поправьте замечания

resp.sendRedirect(req.getContextPath() + "/game?step=" + nextStepId);

}

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Класс совмещает логику обработки HTTP-запросов, валидацию данных, управление сессией, бизнес-логику определения следующего шага и обновление статистики. Следует выделить отдельный сервисный класс GameLogicService, который будет отвечать за обработку ответов игрока и определение следующего шага

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants